hooks/hackathon/use-delete-hackathon.ts (1)

19-25: Use const arrow style for this hook export to match TS conventions in this repo.

The hook is currently a function declaration; convert it to a const arrow with an explicit return type for consistency.

As per coding guidelines **/*.{ts,tsx}: “Prefer const arrow functions with explicit type annotations over function declarations”.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@hooks/hackathon/use-delete-hackathon.ts` around lines 19 - 25, Convert the
function declaration export function useDeleteHackathon({...}:
UseDeleteHackathonOptions) to a const arrow export: declare const
useDeleteHackathon: (opts: UseDeleteHackathonOptions) => ReturnType and assign
an arrow function to it, keeping the same parameter destructuring and body;
ensure you add an explicit return type (the hook’s return type used elsewhere in
the codebase) and keep the existing export behavior so references to
useDeleteHackathon continue to work.

lib/api/hackathons/draft.ts (1)

41-45: Avoid duplicating DeleteHackathonResponse across API modules.

This interface is already defined in lib/api/hackathons/core.ts and is structurally identical. Keeping one shared type prevents silent drift.

Proposed refactor

 import api from '../api';
 import { ApiResponse, PaginatedResponse } from '../types';
 import type { HackathonDraft, Hackathon } from '@/types/hackathon';
+import type { DeleteHackathonResponse } from './core';
@@
-export interface DeleteHackathonResponse extends ApiResponse<null> {
-  success: true;
-  message: string;
-  data: null;
-}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@lib/api/hackathons/draft.ts` around lines 41 - 45, Duplicate interface
DeleteHackathonResponse exists in this module; delete the local declaration and
instead import and reuse the single exported DeleteHackathonResponse from the
core module where it’s defined, or re-export that core type if this module needs
to expose it; ensure any references in this file use the imported type and
update the module exports accordingly to avoid type duplication and drift.

app/(landing)/organizations/[id]/hackathons/page.tsx (1)

167-191: Refactor handleDeleteClick to use early returns and reduce nesting.

The branching works, but flattening this with early returns will make the control flow easier to maintain.

Proposed refactor

 const handleDeleteClick = (
   hackathonId: string,
   type: 'draft' | 'hackathon'
 ) => {
-  if (type === 'hackathon') {
-    const published = publishedHackathons.find(h => h.id === hackathonId);
-    if (published) {
-      setHackathonToDelete({
-        id: hackathonId,
-        title: published.name || 'Untitled Hackathon',
-        type: 'hackathon',
-      });
-      setDeleteDialogOpen(true);
-    }
-  } else {
-    const draft = draftHackathons.find(d => d.id === hackathonId);
-    if (draft) {
-      setHackathonToDelete({
-        id: hackathonId,
-        title: draft.data.information?.name || 'Untitled Hackathon',
-        type: 'draft',
-      });
-      setDeleteDialogOpen(true);
-    }
-  }
+  if (type === 'hackathon') {
+    const published = publishedHackathons.find(h => h.id === hackathonId);
+    if (!published) return;
+    setHackathonToDelete({
+      id: hackathonId,
+      title: published.name || 'Untitled Hackathon',
+      type: 'hackathon',
+    });
+    setDeleteDialogOpen(true);
+    return;
+  }
+
+  const draft = draftHackathons.find(d => d.id === hackathonId);
+  if (!draft) return;
+  setHackathonToDelete({
+    id: hackathonId,
+    title: draft.data.information?.name || 'Untitled Hackathon',
+    type: 'draft',
+  });
+  setDeleteDialogOpen(true);
 };

As per coding guidelines **/*.{js,jsx,ts,tsx}: “Use early returns to improve code clarity”.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@app/`(landing)/organizations/[id]/hackathons/page.tsx around lines 167 - 191,
Refactor handleDeleteClick to use early returns: first branch on the type (if
type === 'hackathon') then find the published item via publishedHackathons.find;
if not found return immediately, otherwise call setHackathonToDelete({ id:
hackathonId, title: published.name || 'Untitled Hackathon', type: 'hackathon' })
and setDeleteDialogOpen(true) and return; for the 'draft' path find the draft
from draftHackathons, return if missing, otherwise call setHackathonToDelete({
id: hackathonId, title: draft.data.information?.name || 'Untitled Hackathon',
type: 'draft' }) and setDeleteDialogOpen(true). Ensure no nested if blocks
remain and keep existing variable names (handleDeleteClick, publishedHackathons,
draftHackathons, setHackathonToDelete, setDeleteDialogOpen).

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

• Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

Run ID: d789430e-1491-4f49-bb1e-5eaeb389d45d